Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release 3.7 #159

Merged
merged 3 commits into from
Nov 14, 2016
Merged

Release 3.7 #159

merged 3 commits into from
Nov 14, 2016

Conversation

jakobw
Copy link
Member

@jakobw jakobw commented Nov 8, 2016

Some patches outside of this repo depend on UnknownForeignRepositoryException so it would be nice to have this released.

@manicki
Copy link
Member

manicki commented Nov 8, 2016

PrefixMappingEntityIdParser has been also added. I am not sure if want to explicitly mention it, or is it OK to only mentioned the factory class.

@jakobw
Copy link
Member Author

jakobw commented Nov 8, 2016

Added it!

@JeroenDeDauw
Copy link
Contributor

If the thing is package public, it should be mentioned. If it's package private, it should not be mentioned, though then its good to mention in the class doc that it is indeed package private (instead of having a since tag)

@manicki
Copy link
Member

manicki commented Nov 9, 2016

-1
Please note that per https://phabricator.wikimedia.org/T150328 DispatchingEntityLookup should be adjusted. This should happen before releasing a new version of this component.

@manicki
Copy link
Member

manicki commented Nov 14, 2016

DispatchingEntityLookup is now ready to be released.

@manicki
Copy link
Member

manicki commented Nov 14, 2016

Added sentence saying that we raised DM version requirement

@manicki manicki merged commit 7aae4fb into master Nov 14, 2016
@manicki manicki deleted the release-3.7 branch November 14, 2016 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants